Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate from Travis to gh-actions #388

Closed

Conversation

mnlevy1981
Copy link
Collaborator

Travis CI doesn't seem to be configured correctly, so the past few PRs were tested by hand. Following the guidance of ESMCI/cime, this PR is attempting to use github actions for continuous integration instead of Travis.

Note that Travis let me rely on a very outdated version of Ubuntu, so I was testing with gfortran 4.8 and an old openmpi version -- on my laptop, the --mpitasks 2 fails at build with

  148 |     call MPI_Bcast(logical_to_bcast, 1, MPI_LOGICAL, root, MPI_COMM_WORLD, ierr)
      |                   1
......
  166 |     call MPI_Bcast(int_to_bcast, 1, MPI_INTEGER, root, MPI_COMM_WORLD, ierr)
      |                   2
Error: Type mismatch between actual argument at (1) and actual argument at (2) (LOGICAL(4)/INTEGER(4)).
  129 |     call MPI_Bcast(str_to_bcast, len(str_to_bcast), MPI_CHARACTER, root, MPI_COMM_WORLD, ierr)
      |                   1
......
  166 |     call MPI_Bcast(int_to_bcast, 1, MPI_INTEGER, root, MPI_COMM_WORLD, ierr)
      |                   2
Error: Type mismatch between actual argument at (1) and actual argument at (2) (CHARACTER(*)/INTEGER(4)).

I found a fix for this (replacing include 'mpif.h' with use mpi), which will be coming in a later commit to this PR but this may mean we can only build the model serially using the much older gfortran (which I believe is still needed for the matlab / python drivers, although I haven't tested that with a newer compiler).

This commit won't build the documentation (yet), but hopefully the Fortran
tests will run (and pass)
Also changes "include 'mpif.h'" to "use mpi" to avoid build errors with most
modern versions of openmpi
Was getting "invalid workflow file" error
eventually the workflow will run, I hope...
@mnlevy1981
Copy link
Collaborator Author

closed in favor of #389

@mnlevy1981 mnlevy1981 closed this Sep 8, 2021
@mnlevy1981 mnlevy1981 deleted the setup_gh_actions branch September 8, 2021 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant